Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix 🐛 Fix ansible #1565

Closed
wants to merge 2 commits into from
Closed

fix 🐛 Fix ansible #1565

wants to merge 2 commits into from

Conversation

kevinwaxi
Copy link

@kevinwaxi kevinwaxi commented Jun 2, 2024 •
edited
Loading

error when setting up for production
"ERROR! [DEPRECATED]: ansible.builtin.include has been removed. Use include_tasks or import_tasks instead. This feature was removed from ansible-core in a release after 2023-05-16. Please update your playbooks."

What type of a PR is this?

  • Bug Fix

Information for review on this pull request:
Error on bench production setup. This caused by depreciated ansible code.

Explain the details for making this change. What existing problem does the pull request solve? The following checklist could help

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you lint your code locally prior to submission?
  • Have you successfully run tests with your changes locally?
  • Does your commit message have an explanation for your changes and why you'd like us to include them?
  • Docs have been added / updated
  • Tests for the changes have been added (for bug fixes / features)
  • Did you modify the existing test cases? If yes, why?

@kevinwaxi
fix 🐛 Fix ansible
678b76f 
error when setting up for production
"ERROR! [DEPRECATED]: ansible.builtin.include has been removed. Use include_tasks or import_tasks instead. This feature was removed from ansible-core in a release after 2023-05-16. Please update your playbooks."
@kevinwaxi
update Created an update to include precommit hooks
5844094 
After reading the guidelines
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 2, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@dineshpanchal93
Copy link

@kevinwaxi any update ?

@kevinwaxi
Copy link
Author

240612_07h01m23s_screenshot
Error still persistent

@dulhaver
Copy link
Contributor

dulhaver commented Jul 16, 2024 •
edited
Loading

as far as I understand @dineshpanchal93 this being unfixed is the root cause for not being able to get TLS running.

I am waiting for this to become possible again but unfortunately don't understand the mechanics being the failed checks.

Is there any way I can help to make this move forward?

@akhilnarang
Copy link
Member

#1563

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dineshpanchal93 dineshpanchal93 dineshpanchal93 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kevinwaxi @dineshpanchal93 @dulhaver @akhilnarang